Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/merge mowali branch #1

Merged
merged 193 commits into from
Oct 13, 2021

Conversation

shashi165
Copy link
Owner

No description provided.

gibaros and others added 30 commits November 8, 2019 17:32
* Issue934-FixSetEnvVarRCAsBooleanInsteadOfString

* Issue934-JestUnitTestChangeUpdate

* WIP

* mockConfig done

* Issue934-AddParseStringInObjectDependencyAndUpdateDeps

* Issue934-UpdatePackageVersionTo8dot4dot1-snapshotAndUpdateDependencies
- Added noAccept parameter
- Upgraded dependencies
* Tests converted to Jest

* Added rules engine jsonpath dynamic fact

* Updated rules engine API. Added jsonpath package to dependencies. Added rules engine tests. Added Jest config. Added example rules.json.

* Updated hapi/subtext to patch vulnerability

* Updated example rules and corresponding tests to reflect real form of payer and payee

* Added switch endpoint to config to obtain payer and payee information for rules engine. Stubbed validateQuoteRequest method in relevant tests. Running rules engine in validateQuoteRequest (but not doing anything with the results yet).

* Tests converted to Jest

* Replaced coverage-check script. Added coverage threshold to jest config. Removed Istanbul config.

* Uninstalled and reinstalled jest to fix vulnerability

* Added junit + config. Added test:junit npm script. Modified circle config to call junit script. Added test results directory to gitignore.

* Modified example rules and corresponding tests to better reflect actual payer and payee facts

* Updated redirect address to redirect fsp. Made the linter happy. Added rule execution and event handling. Removed empty quote validation, replaced with rules engine execution. Removed redundant setImmediate. New test for INVALID_QUOTE_REQUEST event. Expanded INVALID_QUOTE_REQUEST unit tests.

* Manual merge from upstream

* Integrated properly the new `model/rules.js` into `model/quotes.js` and updated the `quotes.tests.js` accordingly.

* Simplified block of code that was unnecessarily using `map`.

* Forwarding event-handler-modified quote request and headers instead of originals

* Fixed tests by poorly mocking handleRuleEvents on the quotes model

* Cleaned a little bit the result of `handleRuleEvents`;
Updated unit tests in order to pass according to latest code changes.

* Deduplicated the functionality of request sending

* Removed `setImmediate` from all places that it was used.

* Use `CreateInternalServerFSPIOPError` instead of `CreateFSPIOPError`

* fixed the name of rules.json

* rename file rules.example.json

* Refactored existing unit tests in order to make them cleaner and avoid duplication;
Added more unit tests for `handleQuoteRequest` to cover all possible paths.

* Refactored `handleQuoteRequest` to make it more readable and avoid duplication;
Cleaned some minor parts of the whole `quotes.js` file.

* Resolved issue with merge

* Added unit tests for `quotesModel.handleRuleEvents`

* Minor update to behaviour and corresponding test update. Minor test change.

* Lint fix

* Added unit tests for `quotesModel.executeRules`

* Added more unit tests to `quotesModel.executeRules` to cover rejected promises of `axios.request`

* Minor refactoring on the unit tests of `RulesEngine` to use `describe` blocks and `it()` instead of `test()` for consistency.

* Added missing dependency that got removed after merging.

* Updated quotes model unit tests to match the new implementation of config.js

* Updated comments in codebase.

* Removed ``^` sign from dependency in package.json
Bumps [npm-check-updates](https://github.com/tjunnone/npm-check-updates) from 3.2.1 to 3.2.2.
- [Release notes](https://github.com/tjunnone/npm-check-updates/releases)
- [Commits](raineorshine/npm-check-updates@v3.2.1...v3.2.2)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Bumps [eslint](https://github.com/eslint/eslint) from 6.6.0 to 6.7.1.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v6.6.0...v6.7.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
- truncateExtension config
- fix issues with standardJS
…es.json`; (#92)

Added a `rules.example.json` file.
- Proper generate request headers when from switch
* Add file logging support in docker, update dependencies

* Add back missing package after merge conflicts
* Simplified rules unit tests

* Updated json-rules-engine

* Updated code to use native json-rules-engine jsonpath syntax

* Made linter happy. Removed jsonpath dependency.
* Added skeleton for fx rules unit tests

* Removed commented tests from original file

* Update fx.test.js
* Add file logging support in docker, update dependencies

* Add back missing package after merge conflicts
* Simplified rules unit tests

* Updated json-rules-engine

* Updated code to use native json-rules-engine jsonpath syntax

* Made linter happy. Removed jsonpath dependency.
* Added skeleton for fx rules unit tests

* Removed commented tests from original file

* Update fx.test.js
* Add anchore image scanning, update circleci config

* Fix config.yml validity issues

* Bump package version, ensure latest deps are installed
* Add unit tests to bring coverage up to 90%+

remove redundant nyc config

Set up dir structure for tests

Set up dir structure for tests

Add inspect util for ease of testing

working on quotes error test

Add istanbul ignore comments for mockgen only files

working on health check mocks

Working on config mocks

Add bulk quotes not implemented tests

Working on health check tests

Working on health check tests

remove unused comments

working on quotes test

working on config default tests

working on utils tests

working on utils tests

find and replace all stack inspection

find and replace all stack inspection

Working on quote tests

Move http into its own library for ease of mocking

Move http into its own library for ease of mocking

fix existing tests once mocking out http

add tests for handleException

add tests for handleException

add tests for handleException

finish getting model testing up to scratch

fix missing conditions on rule engine tests

Add tests for http

refactor start script to improve tests

work on server testing

working on database mocking

working on knex mocks

working on knex mocks

working on knex mocks

working on knex mocks

working on knex mocks

replace err.stack || util.inspect(err) with getStackOrInspect

work on quite tests

work on quite tests

finish work on cachedDatabase

update dependencies, bump package version to 8.7.0

* remove unneeded test files

* run standard --fix

* bump package version to 8.7.1-snapshot

* bump package version to 8.7.2-snapshot
@shashi165 shashi165 merged commit 395c216 into shashi165:master Oct 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.